Skip to content

Conversation

alexwolfmsft
Copy link
Contributor

@alexwolfmsft alexwolfmsft commented Sep 23, 2025

azure-sdk and others added 2 commits September 23, 2025 16:25
* Fix links

* Update docs/core/porting/github-copilot-app-modernization-overview.md

Co-authored-by: Bill Wagner <[email protected]>

---------

Co-authored-by: Bill Wagner <[email protected]>
@dotnetrepoman dotnetrepoman bot added this to the September 2025 milestone Sep 23, 2025
@BillWagner BillWagner disabled auto-merge September 23, 2025 17:23
@BillWagner BillWagner closed this Sep 23, 2025
auto-merge was automatically disabled September 23, 2025 17:24

Pull request was closed

@BillWagner BillWagner reopened this Sep 23, 2025
@BillWagner BillWagner merged commit 1ee1ed3 into live Sep 23, 2025
17 of 18 checks passed
Copy link
Contributor

@adegeo adegeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Things I noticed

---

# What is GitHub Copilot app modernization
# GitHub Copilot app modernization
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The style guide says to use "What is"
image

It should have a ? though

Suggested change
# GitHub Copilot app modernization
# What is GitHub Copilot app modernization?

href: ../core/porting/github-copilot-app-modernization-overview.md
- name: FAQ
href: ./migration/appmod/faq.md
href: ../core/porting/github-copilot-app-modernization-faq.yml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want to remain in this TOC, you'll have to add the query string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants